Skip to content

Remove RHOSO-XXX IDs from the FVTs #278

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Remove RHOSO-XXX IDs from the FVTs #278

wants to merge 1 commit into from

Conversation

@myadla
Copy link
Contributor

@myadla myadla commented Jun 9, 2025

Remove all RHOSO-XXX IDs from the FVTs and replace them with "TEST"
The RHOSO-XXX IDs are removed, since all RHOSO tests use XML for reporting to polarion

elfiesmelfie
elfiesmelfie requested changes Jun 9, 2025
View reviewed changes
@myadla myadla requested a review from ayefimov-1 June 17, 2025 20:22
@myadla myadla force-pushed the replace_rhosoids branch from eaffd0d to 1222a92 Compare June 17, 2025 20:58
@myadla myadla requested a review from elfiesmelfie June 17, 2025 21:01
@myadla
Copy link
Contributor Author

myadla commented Jun 18, 2025

recheck

@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jun 18, 2025

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 278,588b464aba5c8abc375a99e429fe6631bb7759a0

@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jun 18, 2025

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 278,280b4ba294de3e8d0657ed9612054f7f951493f5

@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jun 18, 2025

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 278,8627a95cc354aa3efd945bfc633b5cbeab186215

@myadla myadla force-pushed the replace_rhosoids branch from 8627a95 to 02e2493 Compare June 18, 2025 15:02
@softwarefactory-project-zuul
Copy link

softwarefactory-project-zuul bot commented Jun 18, 2025

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/infrawatch/feature-verification-tests for 278,02e2493f0bb3420d17800cdae9575cad88d22f67

@myadla myadla marked this pull request as draft June 18, 2025 15:04
elfiesmelfie
elfiesmelfie requested changes Jun 26, 2025
View reviewed changes
Copy link
Collaborator

@elfiesmelfie elfiesmelfie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restore the files that were removed.

Update the roles/common/README to remove mention of these test_id vars.
Update roles/common/main.yml to remove the uses of the test_id vars

ci/report_result.yml
Copy link
Collaborator

@elfiesmelfie elfiesmelfie Jun 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

restore this file

callback_plugins/custom_junit.py
Copy link
Collaborator

@elfiesmelfie elfiesmelfie Jun 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restore this file

ci/vars/common.yml
Copy link
Collaborator

@elfiesmelfie elfiesmelfie Jun 26, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

restore this file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@elfiesmelfie elfiesmelfie elfiesmelfie requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@myadla @elfiesmelfie